Skip to content

Port SPARK-31155 to dev/lint-python script#1346

Merged
HyukjinKwon merged 1 commit intodatabricks:masterfrom
HyukjinKwon:sync-lint-python
Mar 17, 2020
Merged

Port SPARK-31155 to dev/lint-python script#1346
HyukjinKwon merged 1 commit intodatabricks:masterfrom
HyukjinKwon:sync-lint-python

Conversation

@HyukjinKwon
Copy link
Member

This PR ports apache/spark#27912 to Koalas.

@HyukjinKwon
Copy link
Member Author

This was merged just 3 hours ago :-).

@itholic
Copy link
Contributor

itholic commented Mar 17, 2020

LGTM

@HyukjinKwon HyukjinKwon merged commit f63e5ee into databricks:master Mar 17, 2020
@codecov-io
Copy link

codecov-io commented Mar 17, 2020

Codecov Report

Merging #1346 into master will decrease coverage by 2.22%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1346      +/-   ##
==========================================
- Coverage   95.25%   93.02%   -2.23%     
==========================================
  Files          34       34              
  Lines        7541     7541              
==========================================
- Hits         7183     7015     -168     
- Misses        358      526     +168
Impacted Files Coverage Δ
databricks/koalas/usage_logging/__init__.py 24.32% <0%> (-71.18%) ⬇️
databricks/koalas/usage_logging/usage_logger.py 50% <0%> (-50%) ⬇️
databricks/koalas/__init__.py 78.72% <0%> (-14.9%) ⬇️
databricks/conftest.py 88.67% <0%> (-7.55%) ⬇️
databricks/koalas/frame.py 93.7% <0%> (-3.02%) ⬇️
databricks/koalas/namespace.py 86.26% <0%> (-1.5%) ⬇️
databricks/koalas/plot.py 93.33% <0%> (-0.96%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 45c325b...14df307. Read the comment docs.

@HyukjinKwon HyukjinKwon deleted the sync-lint-python branch September 11, 2020 07:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants